Skip to content

[BDMS-692] Fix CD preview comment#235

Merged
TylerAdamMartinez merged 1 commit intostagingfrom
TAM-BDMS-692-branch-preview-comment-fix
Apr 2, 2026
Merged

[BDMS-692] Fix CD preview comment#235
TylerAdamMartinez merged 1 commit intostagingfrom
TAM-BDMS-692-branch-preview-comment-fix

Conversation

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem/context:

  • Preview deployments were not easily accessible to reviewers during PR validation

How

Implementation summary - the following was changed/added/removed:

  • Fixed variable interpolation in the PR comment body (removed quoted EOF so $PREVIEW_URL resolves correctly)

Notes

Any special considerations, workarounds, or follow-up work to note?

  • The workflow relies on SANITIZED_BRANCH naming matching the deployed Cloud Run service

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026

Preview Deployment

Preview URL: https://preview-tam-bdms-692-branch-preview-comment-fix-o-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints and has auth disabled for testing.

GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: |
gh pr comment ${{ github.event.pull_request.number }} --body "$(cat <<'EOF'
gh pr comment ${{ github.event.pull_request.number }} --body "$(cat <<EOF
Copy link
Copy Markdown
Contributor Author

@TylerAdamMartinez TylerAdamMartinez Apr 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the quotes around EOF in the heredoc block to ensure the environment variable is expanded correctly.

@TylerAdamMartinez TylerAdamMartinez merged commit 49a18f6 into staging Apr 2, 2026
2 of 4 checks passed
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026

Preview Deployment

Preview URL: https://preview-tam-bdms-692-branch-preview-comment-fix-o-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints and has auth disabled for testing.

@TylerAdamMartinez TylerAdamMartinez deleted the TAM-BDMS-692-branch-preview-comment-fix branch April 3, 2026 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants